From a2fab4fce5401b510d237fd12d12a5382c5091b0 Mon Sep 17 00:00:00 2001 From: Ian Jackson Date: Fri, 25 Jun 2010 15:13:11 +0100 Subject: [PATCH] xend: Fix up check "mac" address sooner change In changeset 21653, dev_type = sxp.name(dev_config) should not have been moved, otherwise, the checking "mac" paragraph is of no use. (The original patch as submitted was correct but I had to make the change manually as it had been mangled.) Signed-off-by: Ian Jackson Signed-off-by Chunyan Liu --- tools/python/xen/xend/XendDomainInfo.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/python/xen/xend/XendDomainInfo.py b/tools/python/xen/xend/XendDomainInfo.py index 91aa7b696a..1432725a11 100644 --- a/tools/python/xen/xend/XendDomainInfo.py +++ b/tools/python/xen/xend/XendDomainInfo.py @@ -833,6 +833,8 @@ class XendDomainInfo: @type dev_config: SXP object (parsed config) """ log.debug("XendDomainInfo.device_create: %s" % scrub_password(dev_config)) + dev_type = sxp.name(dev_config) + if dev_type == 'vif': for x in dev_config: if x != 'vif' and x[0] == 'mac': @@ -840,7 +842,6 @@ class XendDomainInfo: log.error("Virtual network interface creation error - invalid MAC Address entered: %s", x[1]) raise VmError("Cannot create a new virtual network interface - MAC address is not valid!"); - dev_type = sxp.name(dev_config) dev_uuid = self.info.device_add(dev_type, cfg_sxp = dev_config) dev_config_dict = self.info['devices'][dev_uuid][1] log.debug("XendDomainInfo.device_create: %s" % scrub_password(dev_config_dict)) -- 2.30.2